Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

fields_blueprint_id_for takes root object into account #300

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

panupan
Copy link

@panupan panupan commented Nov 28, 2013

This fixes link_to_add when we have two nested forms on a page with different objects and they have overlapping associations. An example being Issue.attachments and Comment.attachments.

This fixes link_to_add when we have two nested forms on a page with different objects and they have overlapping associations. An example being Issue.attachments and Comment.attachments.
@nunosilva800
Copy link

@panupan @ryanb @lest
I encountered a similar problem and I made this fix:
nunosilva800/nested_form@ryanb:master...master
It seems more simple and it relies on the fact that the blueprint is placed inside of the form (as a first child).

If you want I can create a PR on this (note that I've only updated the jQuery javascript).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants